Skip to content

chore: use java-graalvm-ci-prod gcp project for running tests #3349

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mpeddada1 merged 5 commits into from
Nov 13, 2024
Merged

chore: use java-graalvm-ci-prod gcp project for running tests #3349

mpeddada1 merged 5 commits into from
Nov 13, 2024

Conversation

@mpeddada1
Copy link
Contributor

@mpeddada1 mpeddada1 commented Nov 7, 2024 •
edited
Loading

This PR migrates the presubmits in sdk-platform-java to java-graalvm-ci-prod.

@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Nov 7, 2024
@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 7, 2024

Stacktrace:

�[0m �[0m�[1m�[31mError: �[0m�[0m�[1mError when reading or editing Project Service : Request `List Project Services java-graalvm-ci-prod` returned error: Batch request and retried single request "List Project Services java-graalvm-ci-prod" both failed. Final error: Failed to list enabled services for project java-graalvm-ci-prod: googleapi: Error 403: Permission denied to list services for consumer container

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 12, 2024 •
edited
Loading

Stacktrace after enabling cloudkms.googleapis.com in java-graalvm-ci-prod:

 ITKmsTest.ensureKmsKeyRingExists:60->getKeyRing:76 » StatusRuntime PERMISSION_DENIED: Permission 'cloudkms.keyRings.get' denied on resource ...` (or it may not exist).

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 13, 2024

/gcbrun

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 13, 2024

Quality Gate Passed Quality Gate passed for 'gapic-generator-java-root'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 13, 2024

Quality Gate Passed Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 13, 2024

/gcbrun

@mpeddada1
Copy link
Contributor Author

mpeddada1 commented Nov 13, 2024

The presubmits running on java-graalvm-ci-prod are now passing.

jinseopkim0
jinseopkim0 reviewed Nov 13, 2024
View reviewed changes
Copy link
Contributor

@jinseopkim0 jinseopkim0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Retrying the failed library_generation test

error: RPC failed; curl 56 GnuTLS recv error (-9): Error decoding the received TLS packet.
error: 2366 bytes of body are still expected

https://github.com/googleapis/sdk-platform-java/actions/runs/11823555090/job/32943111452?pr=3349

suztomo
suztomo approved these changes Nov 13, 2024
View reviewed changes
Copy link
Member

@suztomo suztomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The required GraalVM checks passed

@burkedavison
Copy link
Member

burkedavison commented Nov 13, 2024

Please don't merge until all the GraalVM checks are passing.

@burkedavison
Copy link
Member

burkedavison commented Nov 13, 2024

Discussed with Mridula. Approved. The failing checks are duplicates and legacy.

@mpeddada1 mpeddada1 merged commit 4c5a43c into main Nov 13, 2024
50 of 55 checks passed
@mpeddada1 mpeddada1 deleted the use-graalvm-ci-project branch November 13, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jinseopkim0 jinseopkim0 jinseopkim0 left review comments

@suztomo suztomo suztomo approved these changes

@burkedavison burkedavison burkedavison approved these changes

Assignees

No one assigned

Labels

size: xs Pull request size is extra small.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mpeddada1 @burkedavison @suztomo @jinseopkim0 @diegomarquezp